-
-
Notifications
You must be signed in to change notification settings - Fork 465
Use uv for managing python version #5009
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use uv for managing python version #5009
Conversation
Instructions and example for changelogPlease add an entry to Example: ## Unreleased
### Features
- Use uv for managing python version ([#5009](https://github.com/getsentry/sentry-java/pull/5009))If none of the above apply, you can opt out of this check by adding |
1eeaf9c to
af4f8c4
Compare
31b0acb to
53ea75b
Compare
| @@ -0,0 +1,3 @@ | |||
| [project] | |||
| name = "javasdk" | |||
| version = "0.0.0" | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing dependencies in pyproject.toml breaks local development
Medium Severity
The pyproject.toml has no dependencies field, and uv.lock only contains the virtual package. However, requirements.txt lists dependencies like requests that are imported by test/system-test-runner.py. When developers use the new devenv setup (uv sync --frozen), the venv is created without these dependencies, causing ImportError when running the test scripts locally.
Additional Locations (1)
|
cursor review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ Bugbot reviewed your changes and found no bugs!
…batchprocessor_from_sentryclient feat(metrics): [Trace Metrics 22] Close and flush `MetricsBatchProcessor` from `SentryClient`
Performance metrics 🚀
|

📜 Description
💡 Motivation and Context
💚 How did you test it?
📝 Checklist
sendDefaultPIIis enabled.🔮 Next steps